Skip to content

pkg/symbolizer: generate x86_64 version#6714

Draft
tarasmadan wants to merge 24 commits intogoogle:masterfrom
tarasmadan:symbolize-self
Draft

pkg/symbolizer: generate x86_64 version#6714
tarasmadan wants to merge 24 commits intogoogle:masterfrom
tarasmadan:symbolize-self

Conversation

@tarasmadan
Copy link
Copy Markdown
Collaborator

@tarasmadan tarasmadan commented Jan 27, 2026 •
edited
Loading

vibecoded and vibetested, not yet reviewed but ready for deep analysis

Benefits:

  1. Fixed function name for some BBs. llvm-symbolizer uses .Ltmp* names sometimes. It makes sense to register llvm-project bug.
$ llvm-symbolizer --obj=../fuzzing-qemu/linux-stable/vmlinux --inlines < pcs_large.in | grep Ltmp | wc -l
583
  1. Peak memory consumption(1.8M BBs) is ~6Gb comparing to ~4Gb in llvm-symbolizer. But the native symbolizer scales almost perfectly because it shares the heavy DWARF index in memory. Running multiple llvm-symbolizer processes (which syzkaller does for throughput) multiplies the memory cost linearly for the private heap portion.
  2. The performance is comparable to what we see in llvm-symbolizer. Approximately 12-16s to generate local cover.html.

@tarasmadan
pkg/symbolizer: generate x86_64 version
b2cf722 
not yet reviewed but ready for deep analysis
@tarasmadan
syzkaller: fix verification scripts lint errors
cf291b3
@tarasmadan
syzkaller: fix compilation error in pkg/cover/backend
70091d0
@tarasmadan
syzkaller: fix linter errors
4e820f2
@tarasmadan
syzkaller: fix final lint errors
f875dc2
@tarasmadan
syzkaller: fix lint errors (round 2)
29d356f
@tarasmadan
syzkaller: fix final final lint errors
8dcbda3
@tarasmadan
syzkaller: fix verification comments style
c772613
@tarasmadan
syzkaller: fix verification comments style (final)
b5191c4
@tarasmadan
syzkaller: fix lint comments (deletion)
112de25
@tarasmadan
syzkaller: fix verification comments (remove fragments)
2b510f1
@tarasmadan
syzkaller: fix lint comment at line 165
832e870
@tarasmadan
syzkaller: fix panic in symbolizer.Make with nil target
a3e8897
@tarasmadan
syzkaller: fallback to addr2line if elf symbolizer fails
bf20e6e
@tarasmadan
pkg/symbolizer: it works and passes all the tests
b02b0e0 
Cleanup is still needed.
@tarasmadan
pkg/symbolizer: it claims to be fast and stable
b84e759
@tarasmadan
pkg/symbolizer: add symbolization logs
0010d7f
@tarasmadan
pkg/symbolizer: propagate kernel path and fix PC forwarding issue
91ed048
@tarasmadan
pkg/symbolizer: fix performance issues
7b77f05 
25s -> 12s cold
2s cold
@tarasmadan
pkg/symbolizer: enable by flag and log report build time
0db00df
@tarasmadan tarasmadan mentioned this pull request Jan 30, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mvanotti mvanotti Awaiting requested review from mvanotti mvanotti will be requested when the pull request is marked ready for review mvanotti is a code owner

@glpesk glpesk Awaiting requested review from glpesk glpesk will be requested when the pull request is marked ready for review glpesk is a code owner

@eepeep eepeep Awaiting requested review from eepeep eepeep will be requested when the pull request is marked ready for review eepeep is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tarasmadan